-
Notifications
You must be signed in to change notification settings - Fork 84
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Additions from ROpenSci Unconference (Spring 2016) #159
Conversation
… for the weather station)
…t we clean with `clean_daily`
Merge branch 'master' of github.com:hrbrmstr/rnoaa # Conflicts: # man/clean_daily.Rd
…itors and bind results together as one dataframe
This pulls data and I think requires a loaded API key, so I don't think it will run without error on CRAN
As a simple solution, we're just going to do this function as a regular function
…with `id` as a factor, not character
radius = 15, | ||
year_min = 2016, | ||
var = c("TAVG", "PRCP")) | ||
stationsDelhi <- unique(rbind_all(stationsDelhi) %>% select(- distance)) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@geanders switch to use bind_rows
instead, as rbind_all
is deprecated, and change in import statement in rnoaa-package
@geanders thanks for the PR - made a few comments for changes |
Thanks so much for taking the time to review. I've made the suggested changes. |
…e aren't chopped off from final data.
thanks @geanders merged |
Thank you so much! I really appreciate you taking the time on this pull request. Would you let me know the next time you submit a version of rnoaa to CRAN? I have a student working on a package that builds on some of these functions, so it will be good to know for getting the dependencies straight in her package code. Best, Brooke From: Scott Chamberlain notifications@github.com thanks @geandershttps://github.com/geanders merged You are receiving this because you were mentioned. |
thanks for all your work on this, and the work of others, i'll add you as a contributor |
This pull requests incorporates work started at the spring's ROpenSci Unconference. Specific elements of this pull request are:
rnoaa
to pull weather data that can then be used in conjunction with air pollution data pulled using OpenAQ ("rnoaa_openaq.RMD").meteo_nearby_stations
function, to identify stations near a latitude-longitude pair for Global Historical Climatology Network daily weather data (defined in "meteo_distance.R" file).meteo_pull_monitors
, to pull Global Historical Climatology Network daily weather data from multiple monitors with a single function call (defined in "helpers_ghcnd.R" file).meteo_show_cache
andmeteo_clear_cache
functions, for cache management (both files in "meteo_cache.r") (other functions we're adding do not rely on these functions).meteo_coverage
function to determine the coverage (percent of non-missing weather observations over dates pulled) for each weather monitor (in "meteo_utils.R" file), with its own method for plotting coverage.vis_miss
function to visualize missingness in a dataframe, including weather dataframes generated withrnoaa